Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

Conversation

@Yuliya110692
Copy link

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

function getMood() {
let isHappy = true;

function getMood(isHappy) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work fixing the problem 👍

} else {
return "I am not happy";
}
}z

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you have a stray 'z' character here


function greaterThan10(num) {
let isBigEnough;
let isBigEnough = num > 10;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done :)

Comment on lines +7 to +10
function isAcceptableUser(userAge, isLoggedIn) {
let isAcceptable = userAge >= 18 && isLoggedIn;
return isAcceptable;
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome 👍

Comment on lines +21 to +29
function applyDiscount(totalPrice) {
let discountSale;
if(totalPrice > 200) {
discountSale = 10;
} else {
discountSale = 5;
}
return totalPrice - (totalPrice * discountSale) / 100;
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link

@DaleCausierIress DaleCausierIress left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants